Review requested:

• @nodejs/gyp

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.62%. Comparing base (c3dd52a) to head (50b8818).
⚠️ Report is 11 commits behind head on main.

@@            Coverage Diff             @@
##             main   #62859      +/-   ##
==========================================
+ Coverage   89.61%   89.62%   +0.01%     
==========================================
  Files         706      706              
  Lines      219136   219137       +1     
  Branches    41981    41980       -1     
==========================================
+ Hits       196376   196403      +27     
+ Misses      14671    14614      -57     
- Partials     8089     8120      +31     

These reference implementations aren't used in any actual quic tests yet

Technically they are currently used, via the fixtures in test/common/quic/test-{client,server}.mjs and tests in test/parallel/test-quic-test-{client,server}.mjs. Those are just placeholder tests really, but to drop this we should drop those too. Imo we should just revert #59946 if we can't realistically use these as a test fixture any more - that means cleaning up the rests and removing the vendored examples entirely if we don't need them. We'll presumably want to replace this somehow for integration testing eventually, but we'll need to find a new solution.

Helpfully none of this is getting run in CI anyway right now (helpful because it's broken). It would be good to fix that separately as we steadily move towards potentially wrapping up QUIC. I won't derail this with that discussion but if anybody on the @nodejs/build team knows the current state of that I'd love to hear it, so we could work towards setting that up in the near future (just building with --experimental-quic and running the tests as normal) to catch issues like this and help with verifying future QUIC PRs.

All that said, @jasnell is currently in the middle of a big batch of new changes for QUIC, including tests, so he may already have plans here that could conflict with this PR. I'll let him chime in there, I think the current state of things is here if anybody wants to take a look, at a glance I don't see a direct conflict there yet myself.

Technically they are currently used, via the fixtures in test/common/quic/test-{client,server}.mjs and tests in test/parallel/test-quic-test-{client,server}.mjs. Those are just placeholder tests really, but to drop this we should drop those too.

We are going to be committed to c++23 build support for V8 in the future, I'd guess for v27 (https://crbug.com/388070065), hence the rationale for just parking this rather than removing it entirely – and leaving the build targets and test wrappers extant means that interested individuals can always invoke them in a custom environment if desired.

If there is an alternative test implementation that's going to be switched over to then that's great as well, but I'm getting a bit bored of manually editing the build config every time I switch branches, and hot-merging this wouldn't preclude it being overwritten by a new solution – it would just unbreak builds in the meantime.

We are going to be committed to c++23 build support for V8 in the future

Ah ok, I wasn't aware of that. Makes sense to me, happy to merge this as a first step towards cleaning up the situation in any case.

Helpfully none of this is getting run in CI anyway right now (helpful because it's broken). It would be good to fix that separately as we steadily move towards potentially wrapping up QUIC. I won't derail this with that discussion but if anybody on the @nodejs/build team knows the current state of that I'd love to hear it, so we could work towards setting that up in the near future (just building with --experimental-quic and running the tests as normal) to catch issues like this and help with verifying future QUIC PRs.

I don't think anyone has asked the build team, so there is no current state.

For a CI with --experimental-quic the choices are:

• New Jenkins CI job. Request via an issue to the Build WG over in nodejs/build.
• GitHub actions job. Anyone can open a pull request to this repository.

The main differences are the Jenkins CI job runs on the donated servers that the Build WG controls -- the trade off being more limited number of people to update if the job needs to be edited. GitHub actions allows more people to edit (the whole collaborator base) but you are at the mercy of GitHub's runner changes (which may not matter for this particular case).

CI: https://ci.nodejs.org/job/node-test-pull-request/72820/